Skip to content

[IMP] endpoint: request content schema#131

Open
ivantodorovich wants to merge 2 commits into
OCA:19.0from
camptocamp:endpoint-request-content-schema
Open

[IMP] endpoint: request content schema#131
ivantodorovich wants to merge 2 commits into
OCA:19.0from
camptocamp:endpoint-request-content-schema

Conversation

@ivantodorovich
Copy link
Copy Markdown

Implement request content validation through an optional schema.

For application/json, we use the JSON Schema (same than openapi)
For application/xml, we use XSD Schema

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @simahawk,
some modules you are maintaining are being modified, check this out!

@ivantodorovich ivantodorovich force-pushed the endpoint-request-content-schema branch from 13eee45 to 97682fc Compare April 30, 2026 19:04
@ivantodorovich ivantodorovich force-pushed the endpoint-request-content-schema branch from 97682fc to c5c2b78 Compare April 30, 2026 19:07
Copy link
Copy Markdown
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

README to be updated

Comment thread endpoint/models/endpoint_mixin.py
@ivantodorovich ivantodorovich marked this pull request as ready for review May 6, 2026 13:52
@ivantodorovich
Copy link
Copy Markdown
Author

@simahawk I added the configure section in the readme

Copy link
Copy Markdown

@Ricardoalso Ricardoalso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically it LGTM

Thanks 🙏

Comment thread endpoint/models/endpoint_mixin.py
Copy link
Copy Markdown

@vvrossem vvrossem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐐

@ivantodorovich ivantodorovich requested a review from simahawk May 18, 2026 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants